-
-
Notifications
You must be signed in to change notification settings - Fork 498
[Youtube] Add linked album for ytmusic searches #1208
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev
Are you sure you want to change the base?
Conversation
|
is this still in progress ? |
|
Kind of, it has been a while so I have to see if the changes even work now. |
|
I added a new method (getPlaylistId) to StreamInfoItem. This PR introduces an useful value for the playlistId only for youtube music searches which is the browseId. |
|
A better idea might be to replace the playlistId with the url of the playlist? |
Stypox
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! Sorry for not reviewing earlier, I somehow didn't notice this PR.
| } | ||
|
|
||
| @Override | ||
| public String getPlaylistId() throws ParsingException { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be getPlaylistUrl() and return the playlist URL, rather than the ID, just like we have getUploaderUrl(). Also, you could also add getPlaylistName() since YouTube music seems to provide that information too.
Adds the linked album to the song when doing a ytmusic search. This is a draft and the current change is not visible. Some questions:
The browseId currently does nothing, hopefully I can try to get the browse endpoint working for ytmusic. I also plan to add a method to get the linked artist as well.